Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

changelog and cleanup #150

Merged
merged 3 commits into from
Jul 7, 2022
Merged

changelog and cleanup #150

merged 3 commits into from
Jul 7, 2022

Conversation

Yurlungur
Copy link
Collaborator

@Yurlungur Yurlungur commented Jul 7, 2022 •
edited
Loading

PR Summary

It's getting a little tough to keep track of all changes that go into the code. This adds a changelog, based on the way we do things in Parthenon. I also cleaned up the docs and READMEs a little while I was at it.

PR Checklist

  • Adds a test for any bugs fixed. Adds tests for new features.
  • Format your changes by using the make format command after configuring with cmake.
  • Document any new features, update documentation for changes made.
  • Make sure the copyright notice on any files you modified is up to date.
  • After creating a pull request, note it in the CHANGELOG.md file

@Yurlungur Yurlungur added the documentation Improvements or additions to documentation label Jul 7, 2022
@Yurlungur Yurlungur self-assigned this Jul 7, 2022
@jhp-lanl
Copy link
Collaborator

jhp-lanl commented Jul 7, 2022

Looks good to me

@Yurlungur Yurlungur merged commit 9885e14 into main Jul 7, 2022
@Yurlungur Yurlungur deleted the jmm/changelog branch July 7, 2022 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhp-lanl jhp-lanl jhp-lanl approved these changes

@junghans junghans Awaiting requested review from junghans

@dholladay00 dholladay00 Awaiting requested review from dholladay00

@brryan brryan Awaiting requested review from brryan

@rbberger rbberger Awaiting requested review from rbberger

@jdolence jdolence Awaiting requested review from jdolence

@lroberts36 lroberts36 Awaiting requested review from lroberts36

@chadmeyer chadmeyer Awaiting requested review from chadmeyer

@sfogerty sfogerty Awaiting requested review from sfogerty

@ktsai7 ktsai7 Awaiting requested review from ktsai7

@mauneyc-LANL mauneyc-LANL Awaiting requested review from mauneyc-LANL

@annapiegra annapiegra Awaiting requested review from annapiegra

Assignees

@Yurlungur Yurlungur

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Yurlungur @jhp-lanl @jonahm-LANL